Skip to content

Aeres-u99 | Adding dockerfile + docker compose for the sync-player#69

Closed
Aeres-u99 wants to merge 3 commits intoLakunake:mainfrom
Aeres-u99:feature/adding-dockercompose
Closed

Aeres-u99 | Adding dockerfile + docker compose for the sync-player#69
Aeres-u99 wants to merge 3 commits intoLakunake:mainfrom
Aeres-u99:feature/adding-dockercompose

Conversation

@Aeres-u99
Copy link

Docker Compose and Dockerfile for Sync Player

Truly thank you for the amazing tool, it works nicely with my pi and allows effective usage.

This pr adds a basic implementation of 2 stage build for sync player, I was unable to figure out on how to use libc6-compat effectively with alpine to reduce the docker image size However, the node-slim image works and seems to be functional enough.

This should make things a lot better

Tested on: aarch64 raspberry pi 4b+

TODO: Have an official registry for the docker images

@Lakunake Lakunake self-assigned this Mar 7, 2026
@Lakunake
Copy link
Owner

Lakunake commented Mar 7, 2026

i have a local build fixing #68 and #70, after i commit it i will try implementing this(i work on multiple machines)

in the meanwhile you can still improve upon this

@Aeres-u99
Copy link
Author

@Lakunake I think it will be better to not invest too much time in fixing libc6-compat to work with alpine image, reason being it will not provide any specific benefit other than 100mb smaller image size

  • For all the files being served, docker is also a partial and slightly more secure solution but yes handling at code level will be even more better.

I think we should be good to merge this, unless you have something else in mind to include with this

Copy link
Owner

@Lakunake Lakunake left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thank you for your help and recommendation, your changes will be edited and built upon for 1.11.0

@Lakunake
Copy link
Owner

Lakunake commented Mar 11, 2026

i will MANUALLY add these changes and tag/credit you in the release and the hosting method in docs

@Lakunake Lakunake closed this Mar 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

moving thumbnail storage to in project for better compatiblity ALL THE FILES ARE BEING SERVED!!!

2 participants